Skip to content

Make the README hero easier to scan at a glance#396

Closed
NagyVikt wants to merge 4 commits intomainfrom
agent/codex/readme-cli-version-and-scanability-2026-04-23-18-38
Closed

Make the README hero easier to scan at a glance#396
NagyVikt wants to merge 4 commits intomainfrom
agent/codex/readme-cli-version-and-scanability-2026-04-23-18-38

Conversation

@NagyVikt
Copy link
Copy Markdown
Collaborator

Automated by gx branch finish (PR flow).

NagyVikt added 4 commits April 23, 2026 19:53
The top of the README buried the CLI version inside one long flat badge strip, so the hero now leads with the product name and two balanced for-the-badge rows that keep the live release, traction, and support signals readable on GitHub. A focused metadata regression now locks those hero cues in place.

Constraint: GitHub README markup must stay static and render without custom CSS
Rejected: Custom screenshot or banner image | would drift from live version, downloads, CI, and scorecard data
Confidence: high
Scope-risk: narrow
Reversibility: clean
Directive: Keep the hero limited to high-signal operator facts; do not collapse it back into one crowded row without updating the metadata test
Tested: node --test test/metadata.test.js
Tested: openspec validate --specs
Not-tested: Rendered GitHub web preview
The code change is already verified and the PR is open, but the finish flow could not reach MERGED because GitHub still requires review and has all required checks queued. Recording that state in the T1 notes keeps the branch handoff truthful instead of implying cleanup completed.

Constraint: Finish state must reflect GitHub merge truth, not local intent
Rejected: Pretend cleanup completed | would misstate the PR lifecycle
Confidence: high
Scope-risk: narrow
Reversibility: clean
Directive: Remove this blocker note only after PR #396 is actually merged and sandbox cleanup is proven
Tested: gh pr view 396 --json mergeStateStatus,reviewDecision,statusCheckRollup,url
Not-tested: eventual CI results or post-merge cleanup
The README lane is still waiting on GitHub gates, but now the blockers are clearer: the configured non-author code owner cannot be resolved as a reviewer, visible collaborators only expose the PR author, and repository auto-merge is disabled. Capturing that evidence on-branch keeps the handoff actionable instead of vague.

Constraint: Finish truth must include current GitHub gate failures
Rejected: Leave the older blocker note as-is | it omitted the failed reviewer and auto-merge probes
Confidence: high
Scope-risk: narrow
Reversibility: clean
Directive: Replace this blocker note only after a real non-author approval exists or repo policy changes
Tested: gh pr edit 396 --add-reviewer recodeecom
Tested: gh pr merge 396 --auto --squash
Tested: gh pr view 396 --json mergeStateStatus,reviewDecision,statusCheckRollup,url
Not-tested: admin-bypass merge paths
The README lane is not only blocked by review policy; GitHub Actions are also broadly stalled for this repository. Capturing the repo-wide queue pattern and the live GitHub status incident keeps the handoff tied to external evidence instead of treating the pending checks as branch-local flakiness.

Constraint: finish notes must separate repo policy blockers from external infrastructure blockers
Rejected: Assume the queue is branch-specific | repo-wide queued runs and GitHub status contradict that
Confidence: high
Scope-risk: narrow
Reversibility: clean
Directive: Remove this incident note only after Actions runs actually leave queued state or the status incident resolves
Tested: gh api repos/recodeee/gitguardex/actions/permissions
Tested: gh run list --limit 20 --json workflowName,status,conclusion,headBranch,createdAt,updatedAt,url
Tested: https://www.githubstatus.com/api/v2/incidents/unresolved.json
Not-tested: eventual post-incident rerun and merge cleanup
@NagyVikt
Copy link
Copy Markdown
Collaborator Author

Blocked finish status as of 2026-04-23 18:03 UTC:

  • Local verification is green: node --test test/metadata.test.js passed 24/24, and the branch worktree is clean.
  • Merge is still blocked by branch protection: REVIEW_REQUIRED, required status checks test (node 18|20|22) plus Analyze (javascript-typescript) (javascript-typescript), and require_last_push_approval=true.
  • .github/CODEOWNERS currently lists @recodeecom @NagyVikt, but gh pr edit 396 --add-reviewer recodeecom fails with GraphQL: Could not resolve user with login 'recodeecom'.
  • gh pr merge 396 --auto --squash fails with GraphQL: Auto merge is not allowed for this repository (enablePullRequestAutoMerge).
  • Required GitHub checks keep re-triggering but remain QUEUED with no steps started.
  • GitHub currently has an unresolved Actions incident: https://www.githubstatus.com/api/v2/incidents/unresolved.json

Unblock path once GitHub recovers: get a valid non-author code-owner approval on this PR, let the required checks actually leave QUEUED, then rerun gx branch finish --branch "agent/codex/readme-cli-version-and-scanability-2026-04-23-18-38" --base main --via-pr --wait-for-merge --cleanup.

@NagyVikt NagyVikt closed this Apr 23, 2026
@NagyVikt NagyVikt deleted the agent/codex/readme-cli-version-and-scanability-2026-04-23-18-38 branch April 23, 2026 18:43
@NagyVikt NagyVikt restored the agent/codex/readme-cli-version-and-scanability-2026-04-23-18-38 branch April 23, 2026 18:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant